Skip to content

Conversation

@yoshman8
Copy link
Contributor

@yoshman8 yoshman8 commented Oct 3, 2025

No description provided.

Copy link
Owner

@PartMan7 PartMan7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, entry.by === target would be needed here (otherwise you're checking whether entry.by exists)

Also PR title seems to be wrong; please fix; PartMan is very serious and would never shitpost

Copy link
Contributor Author

@yoshman8 yoshman8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done :)

Copy link
Owner

@PartMan7 PartMan7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, looks good!

Will fix the obviously incorrect title

@PartMan7 PartMan7 merged commit 85ecffb into PartMan7:main Oct 3, 2025
4 checks passed
@PartMan7 PartMan7 changed the title Fix parts scrabbledex shenanigans chore: Revert balance changes Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants